Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add CI/CD architecture documentation #9202

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Conversation

gustavovalverde
Copy link
Member

Motivation

Our CI/CD infrastructure is complex and was lacking proper documentation, making it challenging for newcomers and contributors to understand the system. This PR adds comprehensive documentation to:

  • Make our CI/CD architecture more accessible to new contributors
  • Document workflow relationships and dependencies
  • Explain infrastructure components and their roles
  • Provide best practices and known limitations

Closes #9112

Specifications & References

Solution

  1. Added detailed CI/CD documentation in .github/workflows/README.md:

    • Table of contents for easy navigation
    • System overview and core infrastructure details
    • Mermaid diagram showing workflow relationships and parallel execution
    • Detailed sections on:
      • Core infrastructure components
      • Workflow organization
      • Test execution strategy
      • Infrastructure details
      • Best practices
      • Known issues
  2. Added CI/CD section to main README.md:

    • Brief overview of CI/CD capabilities
    • Strategic placement between Getting Started and Documentation sections
    • Direct link to detailed documentation
  3. Documentation improvements:

    • Clear explanation of patch workflows and their rationale
    • Detailed infrastructure dependencies and requirements
    • Comprehensive coverage of test execution patterns
    • External contributor considerations and limitations

PR Author's Checklist

  • The PR name will make sense to users.
  • The PR provides a CHANGELOG summary.
  • The solution is tested.
  • The documentation is up to date.
  • The PR has a priority label.

PR Reviewer's Checklist

  • The PR Author's checklist is complete.
  • The PR resolves the issue.

This commit adds detailed documentation of our CI/CD architecture to help
newcomers and contributors better understand our infrastructure. The changes
include:

- New CI/CD architecture documentation in .github/workflows/README.md with:
  - System overview and infrastructure details
  - Detailed Mermaid diagram showing workflow relationships
  - Core infrastructure components and their roles
  - Test execution strategy and best practices
  - Known issues and external contributor limitations

- Added CI/CD section to main README.md for better discoverability
  - Quick overview of CI/CD capabilities
  - Direct link to detailed documentation

Closes #9112
@gustavovalverde gustavovalverde added A-docs Area: Documentation C-design Category: Software design work A-devops Area: Pipelines, CI/CD and Dockerfiles P-Medium ⚡ labels Feb 4, 2025
@gustavovalverde gustavovalverde self-assigned this Feb 4, 2025
@gustavovalverde gustavovalverde requested review from a team as code owners February 4, 2025 09:38
@gustavovalverde gustavovalverde requested review from upbqdn and removed request for a team February 4, 2025 09:38
@github-actions github-actions bot added the C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG label Feb 4, 2025
@oxarbitrage oxarbitrage added the do-not-merge Tells Mergify not to merge this PR label Feb 4, 2025
Copy link
Contributor

mergify bot commented Feb 4, 2025

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #9202 has been dequeued. The pull request rule doesn't match anymore

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@gustavovalverde gustavovalverde merged commit 6f24c9c into main Feb 4, 2025
95 of 97 checks passed
@gustavovalverde gustavovalverde deleted the docs-ci-cd branch February 4, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-docs Area: Documentation C-design Category: Software design work C-trivial Category: A trivial change that is not worth mentioning in the CHANGELOG do-not-merge Tells Mergify not to merge this PR P-Medium ⚡
Projects
None yet
Development

Successfully merging this pull request may close these issues.

devops: CI architecture is not documented
3 participants